Skip to content

Add cicd-ext CI configuration#59

Open
msimberg wants to merge 40 commits intoghex-org:mainfrom
msimberg:cicd-ext
Open

Add cicd-ext CI configuration#59
msimberg wants to merge 40 commits intoghex-org:mainfrom
msimberg:cicd-ext

Conversation

@msimberg
Copy link
Copy Markdown
Collaborator

@msimberg msimberg commented Mar 25, 2026

No description provided.

Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

1 similar comment
@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@boeschf
Copy link
Copy Markdown
Collaborator

boeschf commented Mar 25, 2026

test comment

1 similar comment
@boeschf
Copy link
Copy Markdown
Collaborator

boeschf commented Mar 25, 2026

test comment

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run

@msimberg
Copy link
Copy Markdown
Collaborator Author

cscs-ci run

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an initial CSCS CI “ci-ext” setup to build backend-specific container images via Spack and run CTest suites selected by labels.

Changes:

  • Labeled CTest tests as serial and parallel-ranks-<n> and adjusted MPI test registration logic.
  • Added CSCS CI pipeline definition to build deps/build images per backend and run labeled tests.
  • Added Spack environment files and Containerfiles to install dependencies and build the project in CI containers.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
test/CMakeLists.txt Adds test labels and introduces an MPIEXEC fallback for parallel C++ tests.
test/bindings/fortran/CMakeLists.txt Adds labels for Fortran parallel tests.
.cscs-ci/spack/ucx.yaml Spack env for backend=ucx +cuda.
.cscs-ci/spack/nccl.yaml Spack env for backend=nccl +cuda (currently unused in pipeline).
.cscs-ci/spack/mpi.yaml Spack env for backend=mpi +cuda.
.cscs-ci/spack/libfabric.yaml Spack env for backend=libfabric +cuda.
.cscs-ci/default.yaml Defines CI jobs to build deps/build images and run labeled serial/parallel tests.
.cscs-ci/container/deps.Containerfile Builds a deps image by concretizing/installing Spack dependencies.
.cscs-ci/container/build.Containerfile Builds the project inside the deps image with backend-specific CMake options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@msimberg
Copy link
Copy Markdown
Collaborator Author

I previously though test_cancel was occasionally flaky in CI, but after running more times I think that was completely wrong. I have still seen the parallel test jobs occasionally hang before any of the tests actually start running and after ctest has discovered tests (or while it's discovering tests). I currently have no explanation for this other than filesystem/network issues. If it continues being a problem I'll have to look closer at it, but at the moment I have no reason to believe it's the tests that are buggy.

@msimberg msimberg requested a review from Copilot April 10, 2026 15:13
@msimberg msimberg marked this pull request as ready for review April 10, 2026 15:13
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants